layout: Exclude hidden actors from struts

Currently we only exclude hidden actors from the input region,
but not from struts. This is an oversight that went unnoticed
until now, probably because we didn't consider the work area
in the overview until recently.

https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/4569

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1963>

diff --git a/js/ui/layout.js b/js/ui/layout.js
index 70ece6cab8870b2f29ef62ca6cbfac5c212b2c83..e3c5f497559c28e1bcb47a3bfa154ceec1f24f46 100644 (file)
--- a/js/ui/layout.js
+++ b/js/ui/layout.js
@@ -974,6 +974,9 @@ var LayoutManager = GObject.registerClass({
 
         for (i = 0; i < this._trackedActors.length; i++) {
             let actorData = this._trackedActors[i];
+            if (!actorData.actor.get_paint_visibility())
+                continue;
+
             if (!(actorData.affectsInputRegion && wantsInputRegion) && !actorData.affectsStruts)
                 continue;
 
@@ -984,7 +987,7 @@ var LayoutManager = GObject.registerClass({
             w = Math.round(w);
             h = Math.round(h);
 
-            if (actorData.affectsInputRegion && wantsInputRegion && actorData.actor.get_paint_visibility())
+            if (actorData.affectsInputRegion && wantsInputRegion)
                 rects.push(new Meta.Rectangle({ x, y, width: w, height: h }));
 
             let monitor = null;